-
-
Notifications
You must be signed in to change notification settings - Fork 6.3k
feat: Add configurable permissions for Actions automatic tokens #36173
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
@lunny @wxiaoguang Please review this |
|
Thank you for asking me to review, but I don't use Actions. You can invite the maintainers from the original issue to review. |
@silverwind Please review |
|
I review mostly frontend stuff and am not much of an actions user myself, so please be patient until someone finds time to review it properly. |
No problem |
|
By the way, I see another (older) PR: Feat/actions token permissions #36113 , it added more than 2000 lines of code. What are the differences? Which PR would win ....... @Zettat123 |
This PR doesn't fully implement the proposal in #24635. (For example, it doesn't support configuring actions access between repositories in the same organization) It seems that #36113 implemented these features, but I think its code needs improvement. |
|
@Zettat123 @silverwind Pls give me a few hours(15-20 hours) and this PR will be ready to go |
But "PR: Feat/actions token permissions #36113" came first, and it is more complete, why not respect the first author, but only review this second one? |
@wxiaoguang should i close my pr ? |
I don't know. Reviewers decide. |
I reviewed both PRs, but did not receive responses to my comments in #36113. If @Excellencedev will address the review comments, I think we should keep this PR. |
|
Imho, the only sensible thing we can do is race these 2 PRs. |
|
Adressed most your comments in my latest commit, now i just need to make sure i fully implement the proposal in #24635 |
|
According to the solution in #24635, I think this PR does not implement:
|
1b550bc to
79c38fe
Compare
|
going to work on this weekend 🎉 |
Signed-off-by: Excellencedev <[email protected]>
|
@ChristopherHX pls review when you have time for a very rigorous review |
|
should be good enough for at least aother review |
|
will remove all the |
|
Thanks, I think the UI is better structured now
|

Overview
This PR introduces granular permission controls for Gitea Actions tokens (
GITEA_TOKEN), aligning Gitea's security model with GitHub Actions standards while maintaining compatibility with Gitea's unique repository unit system.It addresses the need for finer access control by allowing administrators and repository owners to define default token permissions, set maximum permission ceilings, and control cross-repository access within organizations.
Key Features
1. Granular Token Permissions
permissions:keyword in workflow and job YAML files (e.g.,contents: read,issues: write).contentsandpackages, with no access to other units.2. Organization & Repository Settings
3. Security Hardening
4. Technical Implementation
action_run_jobtable. This ensures the token's authority is deterministic throughout the job's lifecycle.contentsscope is applied first, allowing granular scopes likecodeorreleasesto override it for precise control.How to Test
go test ./services/actions/...andgo test ./models/repo/...to verify parsing logic and permission clamping.tests/integration/actions_job_token_test.gocovering:permissions:keyword evaluation.GITEA_TOKENcapabilities.Documentation
Added a PR in gitea's docs for this : https://gitea.com/gitea/docs/pulls/318
/fixes #24635
/claim #24635